Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR introduces land area constraints for wind systems by adding an acres_per_kw input parameter to the WindInputs model. This feature parallels the existing acres_per_kw implementation for PV systems, allowing REopt to constrain wind system sizing based on available land area for systems larger than 1500 kW.
Key changes:
- Added
acres_per_kwfield toWindInputsmodel with a default of 0.03 acres/kW and validation range of 0-100 - Created database migration to support the new field
- Updated test data to include the new parameter
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| reoptjl/models.py | Added acres_per_kw FloatField to WindInputs model with validators and help text |
| reoptjl/migrations/0112_windinputs_acres_per_kw.py | Database migration to add the new field to the schema |
| reoptjl/test/posts/all_inputs_test.json | Updated comprehensive test data to include the new parameter |
| CHANGELOG.md | Documented the new feature in v3.17.3 release notes |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
adfarth
approved these changes
Dec 9, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces a new input parameter for wind systems called
acres_per_kw, which specifies the land area required per kW of wind capacity. This value defaults to 0.03 and only constrains wind system sizing for capacities greater than 1500 kW. The change is reflected in the model, migration, documentation, and test data.Wind system land area constraint
acres_per_kwfield to theWindInputsmodel inreoptjl/models.py, with a default value of 0.03, validation, and help text explaining its use for systems over 1500 kW.0112_windinputs_acres_per_kw.pyto add the new field to the database schema.reoptjl/test/posts/all_inputs_test.jsonto include the newacres_per_kwinput with its default value.Documentation
acres_per_kwinput in theCHANGELOG.mdunder v3.17.3, describing its purpose and default value.